Someone is attempting to deploy a commit to the React Component Team on Vercel.

A member of the Team first needs to authorize it.

Summary of Changes

Hello @luozz1994, I'm Gemini Code Assist¹! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

此拉取请求旨在解决 Modal 组件在首次打开时焦点定位不准确的问题。通过在 lockFocus 机制中引入一个前置检查，确保目标元素在焦点同步之前已被正确聚焦，从而提升了用户体验和可访问性，避免了用户需要二次交互才能使焦点正常工作的困扰。

Highlights

• 解决 Modal 首次聚焦问题: 修复了 Modal 弹框首次聚焦不正确的问题，之前需要第二次操作才能正常聚焦。
• 改进 lockFocus 逻辑: 在 lockFocus 函数中，增加了对当前元素是否已聚焦的判断。如果未聚焦，则会先进行聚焦操作，然后再执行 syncFocus 方法，确保焦点行为的即时性和准确性。

🧠 New Feature in Public Preview: You can now enable Memory to help Gemini Code Assist learn from your team's feedback. This makes future code reviews more consistent and personalized to your project's style. Click here to enable Memory in your admin console.

Walkthrough

在 src/Dom/focus.ts 中：1) 调整了 syncFocus 的选择逻辑——当存在 lastElement 且未获得焦点时，优先选用 lastFocusElement（若在可聚焦列表中），否则仅在 lastElement 本身可聚焦时才使用它，最终才回退到第一个可聚焦元素；2) 为 lockFocus 添加了在元素尚未聚焦时立即调用 element.focus({ preventScroll: true }) 的行为以同步聚焦。新增/更新了相应的单元测试。

Changes

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 分钟

Possibly related PRs

• feat: add ignored element support for focus lock #727：在相同文件 src/Dom/focus.ts 上修改 syncFocus 与 lockFocus 行为，代码级变更高度相关。
• Prevent scroll on focus for matched element #726：涉及在 focus 调用中使用 preventScroll: true 的改动，与本次在 lockFocus 中添加相同选项的实现相似。
• feat: Support useLockFocus #712：引入/修改了 lockFocus / useLockFocus 的实现，与本次新增测试与行为调整有直接联系。

诗歌

🐰 我在光里轻跳，耳朵贴着键盘声，
先把焦点抱紧，再把滚动放轻，
锁住一瞬，放手也有序，
小小窗里梦安宁，
我用胡须记住这一行代码。

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

_{Comment @coderabbitai help to get the list of available commands and usage tips.}